Skip to content

Add Remotes section for romic#47

Merged
johanna0321 merged 2 commits into
mainfrom
issue-46
Jan 15, 2026
Merged

Add Remotes section for romic#47
johanna0321 merged 2 commits into
mainfrom
issue-46

Conversation

@johanna0321

Copy link
Copy Markdown
Contributor

Description of your changes

Add romic to Remotes install to stop claman download errors

Issue ticket number and link

#46

Type of change

  • Bug fix
  • New feature
    • Backwards Incompatible?
  • Refactoring / code clean-up
  • Documentation add / update
  • Automated Test
  • Other (please specify)

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds the romic package to the Remotes section in DESCRIPTION and updates the README with improved installation instructions to address download errors when installing claman.

Changes:

  • Added Remotes section to DESCRIPTION specifying calico/romic as a GitHub dependency
  • Updated README installation instructions to include Bioconductor package installation as a prerequisite step

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
DESCRIPTION Added Remotes section specifying the GitHub source for the romic dependency
README.md Enhanced installation instructions with separate steps for Bioconductor packages and claman installation

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@PMSeitzer PMSeitzer left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@johanna0321 johanna0321 merged commit 7073372 into main Jan 15, 2026
7 checks passed
@johanna0321 johanna0321 deleted the issue-46 branch January 15, 2026 23:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants